src/test/java/com/example/solidconnection/community/comment/service/CommentServiceTest.java (1)

85-91: 테스트 커버리지 강화 확인

1. 테스트명 개선: "생성시간 기준으로 정렬해"를 추가하여 테스트 의도를 명확히 했습니다.
2. 테스트 데이터 확대: 단일 자식 댓글에서 3개의 자식 댓글로 확장하여 정렬 동작을 더 잘 검증할 수 있게 되었습니다.

다만 현재 테스트는 각 댓글의 존재 여부는 확인하지만, 실제 반환 순서는 검증하지 않습니다(lines 109-133에서 filteredOn 사용). CommentRepository의 ORDER BY path, created_at 변경사항을 완전히 검증하려면 다음과 같은 순서 검증을 추가하는 것을 고려해보세요:

() -> assertThat(responses)
    .extracting(PostFindCommentResponse::id)
    .containsExactly(
        parentComment.getId(),
        childComment1.getId(),
        childComment2.getId(),
        childComment3.getId()
    )

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/main/java/com/example/solidconnection/common/BaseEntity.java (1)

11-11: 중복 임포트 정리 권장

Line 4에서 ChronoUnit.MICROS를 이미 정적 임포트했으므로, Line 11의 비정적 임포트는 불필요합니다. Line 40에서 ChronoUnit.MICROS 대신 MICROS를 사용하면 정적 임포트만으로 충분합니다.

다음과 같이 정리할 수 있습니다:

-import java.time.temporal.ChronoUnit;
-
 import lombok.Getter;

그리고 Line 40을:

-    public void setCreatedAt(ZonedDateTime createdAt) {
-        this.createdAt = createdAt.truncatedTo(ChronoUnit.MICROS);
-    }
+    public void setCreatedAt(ZonedDateTime createdAt) {
+        this.createdAt = createdAt.truncatedTo(MICROS);
+    }

src/test/java/com/example/solidconnection/community/post/fixture/PostFixtureBuilder.java (1)

81-92: 코드 중복 제거 및 매개변수 명명 개선 권장

1. 코드 중복:

   • Lines 82-89는 Lines 70-77과 동일합니다.
   • create() 메서드를 private 헬퍼로 추출하고 두 public 메서드에서 재사용하는 것을 권장합니다.

2. 매개변수 명명:

   • int time은 단위가 불명확합니다. Line 90에서 plusSeconds(time)로 사용되므로 int secondsOffset 또는 int delaySeconds가 더 명확합니다.

다음과 같이 리팩토링할 수 있습니다:

-    public Post create() {
+    private Post buildPost() {
         Post post = new Post(
                 title,
                 content,
                 isQuestion,
                 likeCount,
                 viewCount,
                 postCategory);
         post.setBoardAndSiteUserId(board.getCode(), siteUser.getId());
+        return post;
+    }
+
+    public Post create() {
+        Post post = buildPost();
         return postRepository.save(post);
     }
 
-    public Post createAfterCertainTime(int time) {
-        Post post = new Post(
-                title,
-                content,
-                isQuestion,
-                likeCount,
-                viewCount,
-                postCategory);
-        post.setBoardAndSiteUserId(board.getCode(), siteUser.getId());
-        post.setCreatedAt(ZonedDateTime.now(ZoneOffset.UTC).plusSeconds(time));
+    public Post createAfterCertainTime(int delaySeconds) {
+        Post post = buildPost();
+        post.setCreatedAt(ZonedDateTime.now(ZoneOffset.UTC).plusSeconds(delaySeconds));
         return postRepository.save(post);
     }

src/test/java/com/example/solidconnection/community/post/fixture/PostFixture.java (1)

51-70: 매개변수 명명 일관성 개선 권장

Line 58의 int time 매개변수는 PostFixtureBuilder의 동일한 문제를 상속합니다. PostFixtureBuilder에서 제안한 대로 int delaySeconds로 변경하면 사용 의도가 명확해집니다.

 public Post 게시글_특정_시간_이후로_저장(
         String title,
         String content,
         Boolean isQuestion,
         PostCategory postCategory,
         Board board,
         SiteUser siteUser,
-        int time
+        int delaySeconds
 ) {
     return postFixtureBuilder
             .title(title)
             .content(content)
             .isQuestion(isQuestion)
             .likeCount(0L)
             .viewCount(0L)
             .postCategory(postCategory)
             .board(board)
             .siteUser(siteUser)
-            .createAfterCertainTime(time);
+            .createAfterCertainTime(delaySeconds);
 }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

src/test/java/com/example/solidconnection/community/comment/service/CommentServiceTest.java (1)

96-97: 디버그 코드 재등장 - 다시 제거 필요합니다.

System.out.println(responses); 코드가 이전 커밋(44ae338~5540b44)에서 제거되었으나 다시 추가되었습니다. 테스트 코드에서 표준 출력은 CI 로그를 오염시키므로 제거해주세요.

-            System.out.println(responses);
-

src/test/java/com/example/solidconnection/common/helper/TestTimeHelper.java (2)

15-17: 예외 처리를 개선하여 디버깅을 용이하게 해주세요.

현재 모든 예외를 RuntimeException으로 감싸면서 원래 예외 메시지가 손실됩니다. 테스트 실패 시 디버깅을 돕기 위해 더 구체적인 메시지를 추가하는 것을 권장합니다.

다음과 같이 개선할 수 있습니다:

         } catch (Exception e) {
-            throw new RuntimeException(e);
+            throw new RuntimeException("Failed to set createdAt on BaseEntity: " + e.getMessage(), e);
         }

---

12-14: 선택사항: 필드 접근성을 원래대로 복원하는 것을 고려해보세요.

setAccessible(true) 호출 후 필드를 다시 setAccessible(false)로 되돌리지 않지만, 테스트 환경에서는 큰 문제가 되지 않습니다. 다만 더 엄격한 격리가 필요하다면 finally 블록에서 복원하는 것도 방법입니다.

더 엄격한 격리가 필요한 경우:

 public static void setCreatedAt(BaseEntity entity, ZonedDateTime time) {
+    Field field = null;
     try {
-        Field field = BaseEntity.class.getDeclaredField("createdAt");
+        field = BaseEntity.class.getDeclaredField("createdAt");
         field.setAccessible(true);
         field.set(entity, time.truncatedTo(ChronoUnit.MICROS));
     } catch (Exception e) {
         throw new RuntimeException("Failed to set createdAt on BaseEntity: " + e.getMessage(), e);
+    } finally {
+        if (field != null) {
+            field.setAccessible(false);
+        }
     }
 }

src/test/java/com/example/solidconnection/community/comment/fixture/CommentFixtureBuilder.java (1)

56-58: 선택사항: 코드 중복을 제거하여 유지보수성을 높여보세요.

Lines 56-58은 createChild() 메서드의 Lines 49-51과 동일합니다. 공통 로직을 추출하면 향후 변경 시 일관성을 유지하기 쉬워집니다.

다음과 같이 리팩토링할 수 있습니다:

     public Comment createChild() {
+        return saveChild();
+    }
+
+    public Comment createChildWithDelaySeconds(long seconds) {
+        Comment saved = saveChild();
+        TestTimeHelper.setCreatedAt(saved, saved.getCreatedAt().plusSeconds(seconds));
+        return commentRepository.save(saved);
+    }
+
+    private Comment saveChild() {
         Comment comment = new Comment(content);
         comment.setPostAndSiteUserId(post, siteUser.getId());
         comment.setParentCommentAndPostAndSiteUserId(parentComment, post, siteUser.getId());
         return commentRepository.save(comment);
     }
-
-    public Comment createChildWithDelaySeconds(long seconds) {
-        Comment comment = new Comment(content);
-        comment.setPostAndSiteUserId(post, siteUser.getId());
-        comment.setParentCommentAndPostAndSiteUserId(parentComment, post, siteUser.getId());
-
-        Comment saved = commentRepository.save(comment);
-
-        TestTimeHelper.setCreatedAt(saved, saved.getCreatedAt().plusSeconds(seconds));
-
-        return commentRepository.save(saved);
-    }

src/test/java/com/example/solidconnection/community/post/service/PostQueryServiceTest.java (1)

109-135: 최신순(내림차순) 시나리오 검증이 명확합니다.

• 1. 메서드명·설명 모두 최신순 의도를 잘 드러냅니다.
• 2. ID 순서까지 containsExactly로 보강한 점이 좋습니다. 다만 createdAt 자체의 내림차순을 직접 검증하는 보조 어서션을 고려해보세요.

다음과 같이 추후 보강을 제안드립니다(필드가 노출된다면):

assertThat(actualResponses)
    .extracting(PostListResponse::createdAt)
    .isSortedAccordingTo(Comparator.reverseOrder());

src/main/java/com/example/solidconnection/community/post/repository/PostRepository.java (2)

17-17: 파생 쿼리로 최신순을 명시한 선택, 다만 페이지네이션도 고려해주세요.

• 1. 대량 데이터에서 전량 List 반환은 부담이 큽니다.
• 2. Pageable/Sort를 받아 확장성 있게 가는 편이 안전합니다.

예시:

Page<Post> findByBoardCodeOrderByCreatedAtDesc(String boardCode, Pageable pageable);
// 또는
List<Post> findByBoardCode(String boardCode, Sort sort);

서비스에서는 Sort.by(DESC, "createdAt") 혹은 Pageable.ofSize(n)로 제어하세요.

---

19-27: NOT IN → NOT EXISTS로 NULL/성능 리스크 완화 제안.

• 1. NOT IN 서브쿼리에 NULL이 섞일 경우 3VL로 인해 예외적 동작을 유발할 수 있습니다.
• 2. NOT EXISTS 패턴이 인덱스 활용·카디널리티 추정에 유리한 경우가 많습니다.

-    @Query("""
-       SELECT p FROM Post p
-       WHERE p.boardCode = :boardCode
-       AND p.siteUserId NOT IN (
-           SELECT ub.blockedId FROM UserBlock ub WHERE ub.blockerId = :siteUserId
-       )
-       ORDER BY p.createdAt DESC
-       """)
+    @Query("""
+       SELECT p FROM Post p
+       WHERE p.boardCode = :boardCode
+         AND NOT EXISTS (
+           SELECT 1 FROM UserBlock ub
+           WHERE ub.blockerId = :siteUserId
+             AND ub.blockedId = p.siteUserId
+         )
+       ORDER BY p.createdAt DESC
+       """)
    List<Post> findByBoardCodeExcludingBlockedUsersOrderByCreatedAtDesc(@Param("boardCode") String boardCode, @Param("siteUserId") Long siteUserId);

추가 제안:

• 3. 목록 화면 N+1 완화: 썸네일 산출에 postImageList가 필요하다면 @entitygraph(attributePaths = "postImageList")를 본 조회에도 고려하세요.
• 4. 운영 팁: 인덱스 (boardCode, createdAt DESC), (UserBlock.blockerId, blockedId) 추천입니다.

src/main/java/com/example/solidconnection/community/post/service/PostQueryService.java (2)

49-62: 정렬 보장은 서비스에서 확실해졌습니다. 다만 필터링을 DB로 내리면 더 안전합니다.

• 1. 최신순 보장을 위해 정렬된 리포지토리를 호출한 점은 적절합니다.
• 2. 카테고리 필터를 메모리에서 하지 말고 DB로 위임하면 불필요 로우 로딩을 줄일 수 있습니다.

리포지토리에 다음 시그니처를 추가하고 여기서 분기 호출하세요.

List<Post> findByBoardCodeAndCategoryOrderByCreatedAtDesc(String boardCode, PostCategory category);
List<Post> findByBoardCodeAndCategoryExcludingBlockedUsersOrderByCreatedAtDesc(String boardCode, Long siteUserId, PostCategory category);

서비스 예시:

-        List<Post> postList;
-        if (siteUserId != null) {
-            postList = postRepository.findByBoardCodeExcludingBlockedUsersOrderByCreatedAtDesc(boardCode, siteUserId);
-        } else {
-            postList = postRepository.findByBoardCodeOrderByCreatedAtDesc(boardCode);
-        }
-        return PostListResponse.from(getPostListByPostCategory(postList, postCategory));
+        List<Post> postList =
+            siteUserId != null
+                ? (postCategory == PostCategory.전체
+                    ? postRepository.findByBoardCodeExcludingBlockedUsersOrderByCreatedAtDesc(boardCode, siteUserId)
+                    : postRepository.findByBoardCodeAndCategoryExcludingBlockedUsersOrderByCreatedAtDesc(boardCode, siteUserId, postCategory))
+                : (postCategory == PostCategory.전체
+                    ? postRepository.findByBoardCodeOrderByCreatedAtDesc(boardCode)
+                    : postRepository.findByBoardCodeAndCategoryOrderByCreatedAtDesc(boardCode, postCategory));
+        return PostListResponse.from(postList);

선택: Pageable/Limit을 추가해 상단 n개만 읽는 패턴으로 최적화할 수 있습니다.

---

93-99: validateCode는 Enum을 직접 반환하는 편이 더 명료합니다.

• 1. String.valueOf(BoardCode.valueOf(code)) 대신 BoardCode를 반환하고 .name()은 호출부에서 사용하세요.
• 2. 타입 안정성과 오타 방지에 유리합니다.

예시:

-    private String validateCode(String code) {
-        try {
-            return String.valueOf(BoardCode.valueOf(code));
+    private BoardCode validateCode(String code) {
+        try {
+            return BoardCode.valueOf(code);
         } catch (IllegalArgumentException ex) {
             throw new CustomException(INVALID_BOARD_CODE);
         }
     }

호출부:

-    String boardCode = validateCode(code);
-    boardRepository.getByCode(boardCode);
+    BoardCode boardCode = validateCode(code);
+    boardRepository.getByCode(boardCode.name());

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Learnt from: Gyuhyeok99
PR: solid-connection/solid-connect-server#522
File: src/main/java/com/example/solidconnection/common/BaseEntity.java:39-41
Timestamp: 2025-09-30T09:16:14.959Z
Learning: BaseEntity와 같은 엔티티의 타임스탬프 필드는 테스트를 위해 public setter를 추가하지 말고, reflection이나 EntityManager를 활용한 테스트 전용 헬퍼를 사용해야 합니다. 프로덕션 코드의 캡슐화를 유지하는 것이 중요합니다.

src/test/java/com/example/solidconnection/community/post/service/PostQueryServiceTest.java (1)

138-159: 순서 검증 로직 추가를 권장합니다.

현재 테스트는 usingRecursiveComparison으로 내용만 비교하고 있습니다. 최신순 정렬을 명시적으로 검증하기 위해 ID 순서 확인을 추가하는 것이 좋습니다.

다음과 같이 순서 검증을 추가해보세요:

         // then
-        assertThat(actualResponses)
-                .usingRecursiveComparison()
-                .ignoringFieldsOfTypes(ZonedDateTime.class)
-                .isEqualTo(expectedResponses);
+        assertAll(
+                () -> assertThat(actualResponses)
+                        .usingRecursiveComparison()
+                        .ignoringFieldsOfTypes(ZonedDateTime.class)
+                        .isEqualTo(expectedResponses),
+                () -> assertThat(actualResponses)
+                        .extracting(PostListResponse::id)
+                        .containsExactly(post4.getId(), post3.getId(), post1.getId())
+        );

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro